old_snapshot: add test for coverage

  • Jump to comment-1
    sh95119@gmail.com2022-08-08T03:54:36+00:00
    Hi hackers, I wrote a test of the old_snapshot extension for coverage. I hope that this is written correctly. before: 0% after: 100% --- regards, Lee Dong Wook.
    • Jump to comment-1
      tgl@sss.pgh.pa.us2022-08-08T05:37:28+00:00
      Dong Wook Lee <sh95119@gmail.com> writes: > I wrote a test of the old_snapshot extension for coverage. Hmm, does this really provide any meaningful coverage? The test sure looks like it's not doing much. I spent some time a week or so ago trying to graft testing of contrib/old_snapshot into src/test/modules/snapshot_too_old. I didn't come up with anything I liked, but I still think that that might lead to a more thorough test than a standalone exercise. regards, tom lane